Skip to content
This repository was archived by the owner on Jul 10, 2024. It is now read-only.

Conversation

@kevin85421
Copy link
Member

@kevin85421 kevin85421 commented Jun 20, 2021

What is this PR for?

As @KUAN-HSUN-LI said in the issue SUBMARINE-838, if we remove our local maven cache (~/.m2/), maven will report as follows.

image-2021-05-27-11-37-20-931

The root cause of the error is that the artifact pentaho-aggdesigner-algorithm:jar:5.1.5-jhyde, one of the dependency jar files of Hive, is not accessible from the central maven repository anymore.

At first, I tried to add new repositories, including Spring Plugins Repository, Spring Lib M Repository, and Conjars Repository, but all of these three repositories are being deprecated. Hence, this artifact is accessible from none of the repositories anymore.

Because the method mentioned above does not work, @jojochuang recommends excluding this missing artifact and making sure the exclusion will not break our Travis tests.

Reference:
(1) apache/hudi#160
(2) apache/hudi#2481
(3) apache/hudi@d02c0e5

What type of PR is it?

[Bug Fix]

Todos

What is the Jira issue?

https://issues.apache.org/jira/projects/SUBMARINE/issues/SUBMARINE-838?filter=allopenissues

How should this be tested?

  • Step1: Local build
rm -r ~/.m2/
mvn clean install -DskipTests

Screenshots (if appropriate)

  • Step1: Local build

截圖 2021-06-20 上午10 51 22

Questions:

  • Do the license files need updating? No
  • Are there breaking changes for older versions? No
  • Does this need new documentation? No

@KUAN-HSUN-LI
Copy link
Member

+1

Copy link
Contributor

@jojochuang jojochuang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kevin85421
Copy link
Member Author

@KUAN-HSUN-LI @jojochuang Thank you for your reviews!

@asfgit asfgit closed this in f24e25b Jun 20, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants